-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated efr32 test runner to support running multiple test binaries #33714
Closed
feasel0
wants to merge
5
commits into
project-chip:master
from
feasel0:feature/efr32--pw_test_runner
Closed
Updated efr32 test runner to support running multiple test binaries #33714
feasel0
wants to merge
5
commits into
project-chip:master
from
feasel0:feature/efr32--pw_test_runner
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PR #33714: Size comparison from 32c568f to 25ebd41 Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
…to use glob, and updated the test runner to use the flash_image parameter for directories
PR #33714: Size comparison from 32c568f to 9e1a79e Full report (47 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32)
|
PR #33714: Size comparison from 32c568f to 6aea2b2 Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
…pw_test_runner
PR #33714: Size comparison from 47097e0 to 55e748a Full report (49 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, tizen)
|
PR #33714: Size comparison from 47097e0 to fa92b5c Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
REPLACED BY: 34769
This PR extends the pw test-runner for efr32 to support multiple binaries by specifying a directory in the
-f
or--flash_image
argument.Steps to build and run:
This PR also removes nl_test infrastructure from the project. Notes about NL dependency reomval:
"${chip_root}/src/test_driver/efr32:nl_test_service.python"
is actually still needed. Thenl
might just be a naming thing, rather than actual NL-specific infrastructure.NlTest
will need to be changed since I think this class is still needed even after NL dependency is removed.To do:
Currently the test runner waits 1 second after flashing and before running the test so that the device can boot. Now that we're running many tests this 1 second becomes significant. It's worth testing how much we can safely reduce this number, if at all.
Add support for wildcards to indicate subset of the tests, like so:
pw_test_runner.py <other arguments> --flash_image out/debug/*Bdx*
Confirm that we can rename all occurrences of "nl_test" in the efr32 directory (10 of them) to "pw_test". I think this is not actually NL infrastructure, it's just an outdated name.